-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PAC and HOSTAPD YANG files. #16036
base: master
Are you sure you want to change the base?
PAC and HOSTAPD YANG files. #16036
Conversation
Please check unit test: ------------------- Test 565: HOSTAPD configuration in global configuration table.--------------------- |
Thank you. Corrected the unit test and uploaded changes. Will move to review from draft as soon as build is through successfully. |
/azpw run Azure.sonic-buildimage |
/AzurePipelines run Azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
Hi Team, |
No, please fix unit test. |
/azpw run Azure.sonic-buildimage |
/AzurePipelines run Azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
/AzurePipelines run Azure.sonic-buildimage |
Commenter does not have sufficient privileges for PR 16036 in repo sonic-net/sonic-buildimage |
} | ||
|
||
leaf mab { | ||
type enumeration { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
} | ||
|
||
leaf dot1x_system_auth_control { | ||
type enumeration { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Container for hostapd global config."; | ||
|
||
list HOSTAPD_GLOBAL_CONFIG_LIST { | ||
key "global"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Follow the below approach for this global table as well.
description "Initial revision."; | ||
} | ||
|
||
typedef port_mode_enumeration { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggest to remove _enumeration suffix in all typedefs.
Why I did it
Added SONiC YANG files for PAC feature (sonic-net/SONiC#1315).
Work item tracking
How I did it
Added PAC_PORT_CONFIG, MAB_PORT_CONFIG, HOSTAPD_GLOBAL_CONFIG tables for PAC configuration.
How to verify it
Compiled sonic_yang_mgmt-1.0-py3-none-any.whl.
Which release branch to backport (provide reason below if selected)
Tested branch (Please provide the tested image version)
Description for the changelog
SONiC YANG files for PAC sonic-net/SONiC#1315
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)